Meetup json output, minor fixes #6339
Conversation
|
I'm going through PRs in prep for this week's edition. I'm not at all qualified to review this one (which is why you didn't ask me to!) but I'm excited for it nonetheless! We'll leave it for @mariannegoldin. :D |
|
Hi all: I still need to review this; thanks for your patience. |
|
Hi @bdillo - manually tested, and works great. Is the intention to run the script over |
for now yes, I'll probably just add a basic check to see if the string "rust" (case insensitive) shows up in the maybe-rust meetup groups |
This fixes some of the event location capitalization issues. The meetup API returns strings like "San francisco", so this fix will format these things a bit more correctly. Also in GB part of the postal code was used as the state, this is ignored now. Additionally, the meetup API returns an empty string if a part of a location is missing, which we now switch to
None.This also adds a way to output the events as json so they can be used in further automation. Currently we're just merging the lists together manually, having a better output format to use will let us create other tools to merge the lists together and save us some time.
Change was tested manually, example json partial output: